Skip to content

Add kahirokunn to operations-reviewers#1777

Merged
knative-prow[bot] merged 1 commit into
knative:mainfrom
kahirokunn:add-kahirokunn-operations-reviewers
Apr 27, 2026
Merged

Add kahirokunn to operations-reviewers#1777
knative-prow[bot] merged 1 commit into
knative:mainfrom
kahirokunn:add-kahirokunn-operations-reviewers

Conversation

@kahirokunn
Copy link
Copy Markdown
Member

@kahirokunn kahirokunn commented Apr 15, 2026

I've been contributing to the Knative ecosystem, mainly around Knative Operator. Through this, I would like to help out more.

If the maintainers are open to it, I'd appreciate the opportunity to contribute as a reviewer for the Operations Working Group.

Thank you for considering.

@knative-prow knative-prow Bot requested review from aliok and evankanderson April 15, 2026 14:08
@knative-prow knative-prow Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 15, 2026
@kahirokunn
Copy link
Copy Markdown
Member Author

kahirokunn commented Apr 15, 2026

@dprotaso @houshengbo 🙏 🙇

@dprotaso
Copy link
Copy Markdown
Member

dprotaso commented Apr 15, 2026

You need to update this file here:

community/peribolos/knative.yaml

Line 454 in a63a453

members:

/approve
/assign @houshengbo

/hold for PR to be ready

Will save lgtm for @houshengbo when the PR is ready

@knative-prow knative-prow Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 15, 2026
@knative-prow
Copy link
Copy Markdown

knative-prow Bot commented Apr 15, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, kahirokunn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 15, 2026
@kahirokunn
Add kahirokunn to operations-reviewers
b43852c 
Signed-off-by: kahirokunn <okinakahiro@gmail.com>
@kahirokunn kahirokunn force-pushed the add-kahirokunn-operations-reviewers branch from a753b13 to b43852c Compare April 22, 2026 10:47
@kahirokunn
Copy link
Copy Markdown
Member Author

kahirokunn commented Apr 22, 2026

@houshengbo @dprotaso Thanks for the review! I've updated peribolos/knative.yaml to add kahirokunn to the Operations Reviewers team as well. PTAL 🙏

@dprotaso
Copy link
Copy Markdown
Member

dprotaso commented Apr 22, 2026

bump @houshengbo

@dprotaso
Copy link
Copy Markdown
Member

dprotaso commented Apr 27, 2026

/hold cancel

@knative-prow knative-prow Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 27, 2026
@dprotaso
Copy link
Copy Markdown
Member

dprotaso commented Apr 27, 2026

/lgtm

@knative-prow knative-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 27, 2026
@knative-prow knative-prow Bot merged commit 0dc8980 into knative:main Apr 27, 2026
5 checks passed
@kahirokunn
Copy link
Copy Markdown
Member Author

kahirokunn commented Apr 27, 2026

Thank you 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aliok aliok Awaiting requested review from aliok

@evankanderson evankanderson Awaiting requested review from evankanderson

Assignees

@dprotaso dprotaso

@houshengbo houshengbo

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kahirokunn @dprotaso @houshengbo